Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Listen to cache event when deleting metadata #41634

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Listen to cache event when deleting metadata #41634

merged 1 commit into from
Nov 21, 2023

Conversation

artonge
Copy link
Contributor

@artonge artonge commented Nov 21, 2023 •
edited
Loading

Supersede #34586
Fix #34424

@artonge artonge mentioned this pull request Nov 21, 2023
Closed
@artonge artonge requested review from ArtificialOwl and come-nc and removed request for ArtificialOwl November 21, 2023 10:38
@artonge artonge self-assigned this Nov 21, 2023
@artonge artonge added the 3. to review Waiting for reviews label Nov 21, 2023
@artonge artonge added this to the Nextcloud 28 milestone Nov 21, 2023
@artonge artonge added bug php Pull requests that update Php code labels Nov 21, 2023
@artonge artonge mentioned this pull request Nov 21, 2023
Closed
29 tasks
@artonge artonge force-pushed the artonge/fix/listen_to_cache_events_for_metadata branch from 38f6463 to b63ddc7 Compare November 21, 2023 12:52
@artonge artonge requested review from a team and icewind1991 and removed request for a team November 21, 2023 12:52
come-nc
come-nc approved these changes Nov 21, 2023
View reviewed changes
Copy link
Contributor

@come-nc come-nc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

half 🐘

@artonge
Copy link
Contributor Author

artonge commented Nov 21, 2023

CI failure unrelated

@artonge artonge merged commit a29495f into master Nov 21, 2023
49 of 50 checks passed
@artonge artonge deleted the artonge/fix/listen_to_cache_events_for_metadata branch November 21, 2023 14:07
@blizzz blizzz mentioned this pull request Nov 22, 2023
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@come-nc come-nc come-nc approved these changes

@ArtificialOwl ArtificialOwl ArtificialOwl approved these changes

@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 was automatically assigned from nextcloud/server-backend

Assignees

@artonge artonge

Labels
3. to review Waiting for reviews bug php Pull requests that update Php code
Projects
None yet
Milestone
Nextcloud 28
Development

Successfully merging this pull request may close these issues.

[Bug]: oc_file_metadata preservation issues with trashbin
3 participants
@artonge @ArtificialOwl @come-nc